9주차 피드백

1. 필수, 선택 미션 모두 진행해주셨네요! 수고하셨습니다. 특별히 피드백 할 내용이 없어보여서 사소한 사항들만 몇 개 찾아봤습니다..!

2. JwtAuthFilter

   isValid(token) 호출 후 getEmail(token) 두 번 모두 JWT를 파싱하는 과정이 발생합니다.

   로직을 약간 수정하면 한 번만 파싱해도 될 것 같아요.

3. 2번 피드백 과정에서 생각난 내용인데 현재 token을 이용해서 email을 파싱하고 이를 이용해서 loadUser~ 하는걸로 봤을 때 식별자로 이메일을 사용한다고 생각했습니다.

   로컬 로그인 상황에서는 이메일에 NOTNULL 설정하면 아무런 문제가 없는데 소셜 로그인의 경우 (특히 카카오) 이메일이 제공되지 않는 상황이 많은걸로 알아요.. 사용자가 동의하지 않을수도 있고

   이메일을 요청하려면 비즈니스앱 + 심사 등의 과정이 필요했던것 같아요.(카카오 기준) 따라서 식별자에 이메일을 사용하는 부분에 대해 한 번 더 생각해볼 필요가 있을 것 같아요.

   → 수정 방향은 식별자를 이메일을 사용하지 않거나 소셜 로그인 가입자가 이메일을 포함하지 않고 가입하는 경우 서비스 자체적으로 이메일을 입력, 인증 해야만 서비스 이용이 가능하게 구현하는 방법이 있을 것 같아요.

그린 너무 깔끔하게 구현해주셨습니다!! 3번 피드백에 대해서는 당장 수정할 필요 없을 것 같고 나중에~ 진짜 프로젝트 할 때 고려하셔도 될 것 같습니다.
시험잘보세용 🍎 ~